Skip to content

Conversation

@jero-odoo
Copy link
Contributor

@jero-odoo jero-odoo commented Dec 8, 2025

Docs task:https://www.odoo.com/odoo/project.task/5347418

Updating Contacts doc to reflect 18 UI changes and content guidelines
Specifically, section on address types and create new contact form

@jero-odoo jero-odoo self-assigned this Dec 8, 2025
@robodoo
Copy link
Collaborator

robodoo commented Dec 8, 2025

Pull request status dashboard

@C3POdoo C3POdoo requested a review from a team December 8, 2025 15:21
@jero-odoo
Copy link
Contributor Author

@larm-odoo this is ready for review. Thanks!

@jero-odoo jero-odoo requested a review from larm-odoo December 8, 2025 15:31
Copy link
Contributor

@larm-odoo larm-odoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @jero-odoo - great updates, just a few notes. First, I saw a bunch of app references not in bold (from the original version) so I would update all of those. The other thing is, I noticed a lot of different/additional fields on a few different runbots. I assume this is due ot localizatoin packages- fiscal and just selecting the country for the database. I would recommend explaining/calling out all the truly universal fields (unrelated to localizations) and have an admonition mentioning additional fields will be visible depending on the localiation - then you can link ot the fiscal localizaiton section. This way, this doc explains all the default fields, and none of the specific ones. This only caught my attention when the Tax ID field and 1099 field were mentioned- which didn't appear in my runbot field, and then I realized those are US related fields. That's the only major issue I saw. Approving now because I know you got this!

Co-authored-by: larm-odoo <121518652+larm-odoo@users.noreply.github.com>
@jero-odoo jero-odoo force-pushed the 18.0-contacts-type-jero branch from 793ec85 to 6ad4019 Compare December 11, 2025 18:49
@jero-odoo
Copy link
Contributor Author

@Felicious this is ready for review. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants